Skip to content

[v17] Add client tools auto update tctl commands#50971

Merged
vapopov merged 1 commit intobranch/v17from
vapopov/add-auto-updates-tctl-commands-v17
Jan 16, 2025
Merged

[v17] Add client tools auto update tctl commands#50971
vapopov merged 1 commit intobranch/v17from
vapopov/add-auto-updates-tctl-commands-v17

Conversation

@vapopov
Copy link
Copy Markdown
Contributor

@vapopov vapopov commented Jan 11, 2025

Backport #47692 to branch/v17

Related: #50966

@vapopov vapopov added backport no-changelog Indicates that a PR does not require a changelog entry labels Jan 11, 2025
@vapopov vapopov marked this pull request as ready for review January 15, 2025 20:04
@github-actions github-actions Bot requested review from r0mant and tcsc January 15, 2025 20:05
@github-actions github-actions Bot added size/md tctl tctl - Teleport admin tool labels Jan 15, 2025
Base automatically changed from vapopov/auth-client-lazy-loading-v17 to branch/v17 January 15, 2025 21:40
@vapopov vapopov requested review from hugoShaka and tigrato January 15, 2025 23:04
* Add client tools auto update tctl commands

* Always print version for watch command
Restrict update empty target version
Rename command to upsert

* Add alias on/off for tools mode
Rename update command to configure

* Add semantic version validation

* Drop watch command for autoupdate

* Replace Upsert with Update/Create
Add format option for output json/yaml

* Change update message

* Use get/set naming for client-tools

* Add mode to response

* Change sub-command help messages
Leave only aliases for enabled/disabled

* Reorganize tctl commands to have commands not required auth client

* Propagate insecure flag with global config to commands by context

* Fix autoupdate command without auth client

* Change commands to enable/disable/target

* Add retry in case of the parallel request

* Add more than one retry
Code review changes

* Update tool/tctl/common/autoupdate_command.go

Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com>

---------

Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com>
@vapopov vapopov force-pushed the vapopov/add-auto-updates-tctl-commands-v17 branch from 361ea7f to b36afe2 Compare January 15, 2025 23:10
@vapopov vapopov added this pull request to the merge queue Jan 16, 2025
Merged via the queue into branch/v17 with commit 1513a14 Jan 16, 2025
@vapopov vapopov deleted the vapopov/add-auto-updates-tctl-commands-v17 branch January 16, 2025 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/md tctl tctl - Teleport admin tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants